feat(web): add repeat icon to recurring events#1506
Open
tibisabau wants to merge 4 commits intoSwitchbackTech:mainfrom
Open
feat(web): add repeat icon to recurring events#1506tibisabau wants to merge 4 commits intoSwitchbackTech:mainfrom
tibisabau wants to merge 4 commits intoSwitchbackTech:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a recurring-event visual indicator in the Calendar UI by displaying a repeat icon inline before event titles when an event is recurring, aligning with issue #954.
Changes:
- Import and render
RepeatIconbefore titles in grid events, all-day events, and someday sidebar events. - Remove the previous “can’t migrate recurring events” UI in
SomedayEventRectangleand replace it with icon-based indication. - Introduce new recurring detection logic in
SomedayEventRectangle(currently inconsistent with other components).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| packages/web/src/views/Calendar/components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx | Adds repeat icon and changes migrate/recurrence UI logic for someday sidebar event rows. |
| packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvent.tsx | Adds repeat icon before title in all-day event rendering. |
| packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.tsx | Adds repeat icon before title in time-grid event rendering. |
packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.tsx
Show resolved
Hide resolved
.../components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx
Outdated
Show resolved
Hide resolved
.../components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx
Show resolved
Hide resolved
packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvent.tsx
Show resolved
Hide resolved
packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.tsx
Show resolved
Hide resolved
tyler-dane
requested changes
Mar 4, 2026
Contributor
tyler-dane
left a comment
tyler-dane
left a comment
There was a problem hiding this comment.
Hey @tibisabau , thanks for the submission! Please address copilot's review and tag me once you think it's in good shape according to the contribution guidelines.
tyler-dane
requested changes
Mar 5, 2026
Comment on lines
-62
to
-67
| ) : ( | ||
| <Flex> | ||
| <StyledRecurrenceText | ||
| onClick={(e) => { | ||
| e.stopPropagation(); | ||
| alert("Can't migrate recurring events"); |
Contributor
There was a problem hiding this comment.
Plz readd this; it's important to stop the user from trying to migrate recurring events, which is a feature we don't support yet
packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.tsx
Outdated
Show resolved
Hide resolved
packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvent.tsx
Outdated
Show resolved
Hide resolved
packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvent.tsx
Show resolved
Hide resolved
.../components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx
Outdated
Show resolved
Hide resolved
.../components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx
Outdated
Show resolved
Hide resolved
.../components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx
Outdated
Show resolved
Hide resolved
...c/views/Calendar/components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/styled.ts
Show resolved
Hide resolved
packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.test.tsx
Outdated
Show resolved
Hide resolved
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Added a repeat icon from @phosphor-icons/react to visually indicate recurring events across all event display components. The icon appears inline before the event title for events with recurrence rules.
Closes #954.
The code in this pull request was generated by GitHub Copilot with the Claude Sonnet 4.5 model.
Checklist if Applicable
yarn test packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.test.tsx packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvents.test.tsx packages/web/src/views/Calendar/components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventContainer.test.tsxyarn prettier . --write